Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: remove entryComponents from DynamicDialogs #13682

Merged
merged 1 commit into from
Sep 17, 2023

Conversation

tomer953
Copy link
Contributor

@tomer953 tomer953 commented Sep 16, 2023

According to the current documentation for Dynamic Dialogs, it is stated that entryComponents are required for dynamic dialogs. However, with Angular 9 and above, using the Ivy rendering engine, entryComponents are generally no longer necessary. This could lead to confusion for developers using newer versions of Angular.

Defect Fixes

Fix #13679

According to the current documentation for Dynamic Dialogs, it is stated that entryComponents are required for dynamic dialogs. However, with Angular 9 and above, using the Ivy rendering engine, entryComponents are generally no longer necessary. This could lead to confusion for developers using newer versions of Angular.
@vercel
Copy link

vercel bot commented Sep 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Sep 16, 2023 2:00pm

@SoyDiego
Copy link
Contributor

Hi thanks for the PR.
Please, edit your message adding Fix #13679 to link this PR to your issue. If PrimeNG team approve, will be close automatically 😊

@tomer953
Copy link
Contributor Author

Hi thanks for the PR.
Please, edit your message adding Fix #13679 to link this PR to your issue. If PrimeNG team approve, will be close automatically 😊

done

@gucal gucal merged commit 452fdbb into primefaces:master Sep 17, 2023
1 of 5 checks passed
@gucal
Copy link
Contributor

gucal commented Sep 17, 2023

Thanks a lot for the contribution!

Best regards!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DynamicDialog - remove entryComponents from docs
3 participants